Skip to content
This repository has been archived by the owner on Feb 20, 2020. It is now read-only.

Add fetch data, export xlsx and send mail #17

Open
wants to merge 16 commits into
base: master
Choose a base branch
from
Open

Add fetch data, export xlsx and send mail #17

wants to merge 16 commits into from

Conversation

lamvh
Copy link
Contributor

@lamvh lamvh commented Apr 17, 2019

config circleci

lamvh and others added 16 commits April 9, 2019 18:43
@lamvh
feat(orders): add order service and create report excel file
564161f
@lamvh
fixbug(orders): update variable name, function name
b125a4e
@lamvh
fixbug(orders): remove server file config
4a147a1
@lamvh
fixbug(orders): update variable name, and remove unnecessary file, co…
2d184d4 
…ntent
@lamvh
feat(mail): add feature export xlsx file from data and send email
7fb43ff
@lamvh
feat(mail): intergrated slack
1f6c4fc 
add intergration slack api 
add comment in .env example
retry send mail 3 times if fail
@lamvh
fixbug(mail) : update gitignore
a478286
@lamvh
Merge branch 'master' into mail
9d85022
@lamvh
chore: remove debug code
eab4965
@lamvh
chore: update slack noti
b32b148
@duongdev
Merge branch 'master' into mail
9e0e099
@lamvh
chore(mail): update syntax es6
58a9d1a
@lamvh
fea(products): fetch product and export xlsx
52ca6b1
@lamvh
feat(category): fetch category, export xlsx and send mail
735036e
@lamvh
Merge branch 'master' into report-product-category
c089ea8
@lamvh
Merge branch 'mail' into report-product-category
318ad05
@lamvh lamvh requested a review from duongdev April 17, 2019 10:47
@duongdev duongdev changed the title Feat(product-category): fetch data, export xlsx and send mail Add fetch data, export xlsx and send mail Apr 20, 2019
@duongdev
Copy link
Member

@vhlam1997 did you write unit tests for this PR?

@lamvh
Copy link
Contributor Author

lamvh commented Apr 20, 2019

i will complete it soon !

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@duongdev duongdev Awaiting requested review from duongdev

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@lamvh @duongdev